Change to toast message in Reports #578

kavithacm · 2023-02-24T21:55:03Z

Description

Change the condition to look for toast message for confirmation when a report is downloaded. This a flaky test, the machine couldn't capture the in progress model because it disappears too quickly

Issues Resolved

#770

Check List

Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

cypress/integration/plugins/reports-dashboards/04-download.spec.js

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

ashwin-pc · 2023-03-01T18:06:27Z

@kavithacm the lint checker and ci workflow are failing.

kavithacm · 2023-03-02T18:03:46Z

@kavithacm the lint checker and ci workflow are failing.

CI failed due to the following error, rerunning it should fix it, as i made a changes only to reporting cypress which shouldn’t cause it to fail

curl: (7) Failed to connect to localhost port 5601 after 1 ms: Connection refused
{"type":"log","@timestamp":"2023-03-01T18:08:23Z","tags":["fatal","root"],"pid":2126,"message":"Error: Cannot find module 'performance-now'\nRequire stack:\n-

for linting, i'm not exactly sure why its failing, I've formatted the file with prettier before i pushed the code

cypress/integration/plugins/reports-dashboards/04-download.spec.js

kavilla · 2023-03-10T10:29:44Z

Is this for 2.x as well?

kavilla · 2023-03-15T20:55:36Z

@kavithacm any comments on the above?

kavithacm · 2023-07-19T20:22:35Z

Is this for 2.x as well?

Yes it should go in for 2.x as well

ashwin-pc · 2023-08-29T19:29:47Z

The CI of failing. Please fix that before merging

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

ashwin-pc · 2023-08-30T19:43:59Z

The snapshot E2E seems to be broken, so going ahead with the merge for now. cc: @kavilla

* Change to toast message in Reports download spec Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update test Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Fix lint Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> --------- Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> (cherry picked from commit 46c9a48)

* Change to toast message in Reports download spec Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update test Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Fix lint Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> --------- Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> (cherry picked from commit 46c9a48) Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>

* Change to toast message in Reports download spec Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update test Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Fix lint Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> --------- Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: [email protected] <[email protected]>

…ain tenant in shortlink (#736) * Test to ensure when a shortlink is copied, tenant is changed and short link is visited. The tenant from the link is visited. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * After link copy, allow tenant change Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Addressing comments, removing function to it's own file and removing unneeded code in before. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Spelling error rectified Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Linting errors resolved Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update cypress/utils/commands.js Co-authored-by: Yulong Ruan <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Changed data-test-subj to getElementByTestId for consitency Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Switch tenant updated to use interceps and should() for improved test performance Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * switch tenant changed to use intercepts and should() for increaced test performance. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Adding commen to explain how the addtion of the should() helps Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Global declaration of the tenant removed. The clearing of session storage is not needed actually, cy.visit() does this for us Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Additition of 'createDashboard' method to allow programatic dashboard creation. Also improvements to the test. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * When testing we are looking for the private tenant. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * check changed to allow for both 'private' and __user__ Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * linting errors resolved Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * merging in changes on remote Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Adding back --headless parameter. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * undoing package-lock changes. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Use library from release tag instead of git reference (#706) The module installed from the OSD test library seems to update or not based on caching issues. Related proposal: opensearch-project/opensearch-dashboards-test-library#36 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * update testSplitTables test (#731) Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739) * [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Remove unnecessary test case Signed-off-by: Tyler Ohlsen <[email protected]> * Optimize getters Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add AD vis augmenter tests Signed-off-by: Tyler Ohlsen <[email protected]> * More refactoring Signed-off-by: Tyler Ohlsen <[email protected]> * More tests Signed-off-by: Tyler Ohlsen <[email protected]> * Add test for AD cleanup scenario Signed-off-by: Tyler Ohlsen <[email protected]> * Set up saved obj test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add reminder TODO Signed-off-by: Tyler Ohlsen <[email protected]> * Add tests regarding saved obj visibility Signed-off-by: Tyler Ohlsen <[email protected]> * Add view events tests Signed-off-by: Tyler Ohlsen <[email protected]> * cleanup Signed-off-by: Tyler Ohlsen <[email protected]> * remove import Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Signed-off-by: [email protected] <[email protected]> * Revert "[Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)" (#748) This reverts commit 07a67d7. Signed-off-by: manasvinibs <[email protected]> Signed-off-by: [email protected] <[email protected]> * Test sample data with multiple data source enabled on local cluster (#756) Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Vis Augmenter] Add tests in OSD & AD plugin (#752) * [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Remove unnecessary test case Signed-off-by: Tyler Ohlsen <[email protected]> * Optimize getters Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add AD vis augmenter tests Signed-off-by: Tyler Ohlsen <[email protected]> * More refactoring Signed-off-by: Tyler Ohlsen <[email protected]> * More tests Signed-off-by: Tyler Ohlsen <[email protected]> * Add test for AD cleanup scenario Signed-off-by: Tyler Ohlsen <[email protected]> * Set up saved obj test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add reminder TODO Signed-off-by: Tyler Ohlsen <[email protected]> * Add tests regarding saved obj visibility Signed-off-by: Tyler Ohlsen <[email protected]> * Add view events tests Signed-off-by: Tyler Ohlsen <[email protected]> * cleanup Signed-off-by: Tyler Ohlsen <[email protected]> * remove import Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * Fix NPE Signed-off-by: Tyler Ohlsen <[email protected]> * Fix bug of non-empty dashboard; remove unnecessary test Signed-off-by: Tyler Ohlsen <[email protected]> * Simplify dashboard creation Signed-off-by: Tyler Ohlsen <[email protected]> * Update undefined check Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix: CVE of tough-cookie and word-wrap (#763) Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: use "overrides" to install desired version of tough-cookie and word-wrap. (#772) * feat: use npm v8 to install and initialize package-lock Signed-off-by: SuZhou-Joe <[email protected]> * feat: add Node version requirement in DEVELOPER_GUIDE Signed-off-by: SuZhou-Joe <[email protected]> --------- Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: [email protected] <[email protected]> * Improve date selection across versions of OUI (#778) Signed-off-by: Miki <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: sync cypress test from notifications-dashboards (#776) (#784) * feat: sync cypress test from notifications-dashboards Signed-off-by: SuZhou-Joe <[email protected]> * feat: rename commands Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: add larger wait time Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: add refresh before delete all notifications configs Signed-off-by: SuZhou-Joe <[email protected]> --------- Signed-off-by: SuZhou-Joe <[email protected]> (cherry picked from commit 81f70c3) Signed-off-by: [email protected] <[email protected]> * [Table Visualizations] Tests cleanup (#785) Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * Bump dependency on opensearch-dashboards-test-library (#790) Signed-off-by: Miki <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Table Visualizations] Test Update (#787) * fix-test * simplify selector --------- Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix discover (#803) Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: [email protected] <[email protected]> * Change to toast message in Reports (#578) * Change to toast message in Reports download spec Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update test Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Fix lint Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> --------- Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix discover (#807) Signed-off-by: Qingyang(Abby) Hu <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: add test cases for remote models (#813) Signed-off-by: Lin Wang <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix workbench download text and csv schema (#814) (#817) Signed-off-by: Joshua Li <[email protected]> (cherry picked from commit 46c0882) Co-authored-by: Joshua Li <[email protected]> Signed-off-by: [email protected] <[email protected]> * update trace analytics cypress tests according to observability changes (#775) * update ftr Signed-off-by: Derek Ho <[email protected]> * revert file Signed-off-by: Derek Ho <[email protected]> * update with trace analytics accordian changes Signed-off-by: Derek Ho <[email protected]> --------- Signed-off-by: Derek Ho <[email protected]> Signed-off-by: [email protected] <[email protected]> * Bump semver from 7.3.7 to 7.5.4 (#822) Bumps [semver](https://github.com/npm/node-semver) from 7.3.7 to 7.5.4. - [Release notes](https://github.com/npm/node-semver/releases) - [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md) - [Commits](npm/node-semver@v7.3.7...v7.5.4) --- updated-dependencies: - dependency-name: semver dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: [email protected] <[email protected]> * [Dashboard] Retry visbuilder dashboard test (#823) Seeing it pass in video replays but occassionally doesn't pass. However, it doesn't pass with security enabled so I'd imagine it could be related to the refreshing of the session. Issue: opensearch-project/OpenSearch-Dashboards#4947 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * Add new tests for alerting dashboards (#832) Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update source element in import_vector_map_tab.spec.js (#844) * Update source element in import_vector_map_tab.spec.js Signed-off-by: Junqiu Lei <[email protected]> * Update source element in import_vector_map_tab.spec.js Signed-off-by: Junqiu Lei <[email protected]> --------- Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: [email protected] <[email protected]> * Use index pattern id to find page in import_vector_map_tab.spec.js (#847) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: [email protected] <[email protected]> * Buffer `waitForLoader` before checking for icon (#857) (#859) Some async calls occur after navigating and sometimes navigating too quickly prevents the calls from having correct data and therefore the homeIcon is in a bad state. This adds the ability to buffer before checking for the homeIcon. Defaulted to 0 ms, but for `yarn cypress:run-with-security` configures WAIT_FOR_LOADER_BUFFER_MS to be 500 ms Issue resolved: opensearch-project/OpenSearch-Dashboards#5028 Signed-off-by: Kawika Avilla <[email protected]> (cherry picked from commit e7c7e59) Co-authored-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * Overwrite removed and moved to after in unit test Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update dashboard.spec.js Signed-off-by: leanneeliatra <[email protected]> * Update dashboard_sample_data_spec.js remove unneeded change Signed-off-by: leanneeliatra <[email protected]> --------- Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Signed-off-by: manasvinibs <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: Qingyang(Abby) Hu <[email protected]> Signed-off-by: Lin Wang <[email protected]> Signed-off-by: Derek Ho <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Co-authored-by: Yulong Ruan <[email protected]> Co-authored-by: Kawika Avilla <[email protected]> Co-authored-by: Sirazh Gabdullin <[email protected]> Co-authored-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Co-authored-by: Manasvini B Suryanarayana <[email protected]> Co-authored-by: Kristen Tian <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Qingyang(Abby) Hu <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Lin Wang <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Joshua Li <[email protected]> Co-authored-by: Derek Ho <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Junqiu Lei <[email protected]>

…ain tenant in shortlink (#736) * Test to ensure when a shortlink is copied, tenant is changed and short link is visited. The tenant from the link is visited. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * After link copy, allow tenant change Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Addressing comments, removing function to it's own file and removing unneeded code in before. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Spelling error rectified Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Linting errors resolved Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update cypress/utils/commands.js Co-authored-by: Yulong Ruan <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Changed data-test-subj to getElementByTestId for consitency Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Switch tenant updated to use interceps and should() for improved test performance Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * switch tenant changed to use intercepts and should() for increaced test performance. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Adding commen to explain how the addtion of the should() helps Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Global declaration of the tenant removed. The clearing of session storage is not needed actually, cy.visit() does this for us Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Additition of 'createDashboard' method to allow programatic dashboard creation. Also improvements to the test. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * When testing we are looking for the private tenant. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * check changed to allow for both 'private' and __user__ Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * linting errors resolved Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * merging in changes on remote Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Adding back --headless parameter. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * undoing package-lock changes. Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Use library from release tag instead of git reference (#706) The module installed from the OSD test library seems to update or not based on caching issues. Related proposal: opensearch-project/opensearch-dashboards-test-library#36 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * update testSplitTables test (#731) Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739) * [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Remove unnecessary test case Signed-off-by: Tyler Ohlsen <[email protected]> * Optimize getters Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add AD vis augmenter tests Signed-off-by: Tyler Ohlsen <[email protected]> * More refactoring Signed-off-by: Tyler Ohlsen <[email protected]> * More tests Signed-off-by: Tyler Ohlsen <[email protected]> * Add test for AD cleanup scenario Signed-off-by: Tyler Ohlsen <[email protected]> * Set up saved obj test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add reminder TODO Signed-off-by: Tyler Ohlsen <[email protected]> * Add tests regarding saved obj visibility Signed-off-by: Tyler Ohlsen <[email protected]> * Add view events tests Signed-off-by: Tyler Ohlsen <[email protected]> * cleanup Signed-off-by: Tyler Ohlsen <[email protected]> * remove import Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Signed-off-by: [email protected] <[email protected]> * Revert "[Vis Augmenter / Feature Anywhere] Add tests in core OSD and AD plugin (#739)" (#748) This reverts commit 07a67d7. Signed-off-by: manasvinibs <[email protected]> Signed-off-by: [email protected] <[email protected]> * Test sample data with multiple data source enabled on local cluster (#756) Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Vis Augmenter] Add tests in OSD & AD plugin (#752) * [Vis Augmenter / Feature Anywhere] Add test suite for vanilla OSD + helper fns for plugins (#725) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Remove unnecessary test case Signed-off-by: Tyler Ohlsen <[email protected]> * Optimize getters Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * [Feature Anywhere / Vis Augmenter] Add test flows for integration with AD plugin (#727) * feature anywhere initial tests Signed-off-by: Jovan Cvetkovic <[email protected]> * Add test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add AD vis augmenter tests Signed-off-by: Tyler Ohlsen <[email protected]> * More refactoring Signed-off-by: Tyler Ohlsen <[email protected]> * More tests Signed-off-by: Tyler Ohlsen <[email protected]> * Add test for AD cleanup scenario Signed-off-by: Tyler Ohlsen <[email protected]> * Set up saved obj test suite Signed-off-by: Tyler Ohlsen <[email protected]> * Add reminder TODO Signed-off-by: Tyler Ohlsen <[email protected]> * Add tests regarding saved obj visibility Signed-off-by: Tyler Ohlsen <[email protected]> * Add view events tests Signed-off-by: Tyler Ohlsen <[email protected]> * cleanup Signed-off-by: Tyler Ohlsen <[email protected]> * remove import Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> * Fix NPE Signed-off-by: Tyler Ohlsen <[email protected]> * Fix bug of non-empty dashboard; remove unnecessary test Signed-off-by: Tyler Ohlsen <[email protected]> * Simplify dashboard creation Signed-off-by: Tyler Ohlsen <[email protected]> * Update undefined check Signed-off-by: Tyler Ohlsen <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix: CVE of tough-cookie and word-wrap (#763) Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: use "overrides" to install desired version of tough-cookie and word-wrap. (#772) * feat: use npm v8 to install and initialize package-lock Signed-off-by: SuZhou-Joe <[email protected]> * feat: add Node version requirement in DEVELOPER_GUIDE Signed-off-by: SuZhou-Joe <[email protected]> --------- Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: [email protected] <[email protected]> * Improve date selection across versions of OUI (#778) Signed-off-by: Miki <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: sync cypress test from notifications-dashboards (#776) (#784) * feat: sync cypress test from notifications-dashboards Signed-off-by: SuZhou-Joe <[email protected]> * feat: rename commands Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: add larger wait time Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: update Signed-off-by: SuZhou-Joe <[email protected]> * feat: add refresh before delete all notifications configs Signed-off-by: SuZhou-Joe <[email protected]> --------- Signed-off-by: SuZhou-Joe <[email protected]> (cherry picked from commit 81f70c3) Signed-off-by: [email protected] <[email protected]> * [Table Visualizations] Tests cleanup (#785) Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * Bump dependency on opensearch-dashboards-test-library (#790) Signed-off-by: Miki <[email protected]> Signed-off-by: [email protected] <[email protected]> * [Table Visualizations] Test Update (#787) * fix-test * simplify selector --------- Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix discover (#803) Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: [email protected] <[email protected]> * Change to toast message in Reports (#578) * Change to toast message in Reports download spec Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update test Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Fix lint Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> --------- Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix discover (#807) Signed-off-by: Qingyang(Abby) Hu <[email protected]> Signed-off-by: [email protected] <[email protected]> * feat: add test cases for remote models (#813) Signed-off-by: Lin Wang <[email protected]> Signed-off-by: [email protected] <[email protected]> * fix workbench download text and csv schema (#814) (#817) Signed-off-by: Joshua Li <[email protected]> (cherry picked from commit 46c0882) Co-authored-by: Joshua Li <[email protected]> Signed-off-by: [email protected] <[email protected]> * update trace analytics cypress tests according to observability changes (#775) * update ftr Signed-off-by: Derek Ho <[email protected]> * revert file Signed-off-by: Derek Ho <[email protected]> * update with trace analytics accordian changes Signed-off-by: Derek Ho <[email protected]> --------- Signed-off-by: Derek Ho <[email protected]> Signed-off-by: [email protected] <[email protected]> * Bump semver from 7.3.7 to 7.5.4 (#822) Bumps [semver](https://github.com/npm/node-semver) from 7.3.7 to 7.5.4. - [Release notes](https://github.com/npm/node-semver/releases) - [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md) - [Commits](npm/node-semver@v7.3.7...v7.5.4) --- updated-dependencies: - dependency-name: semver dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: [email protected] <[email protected]> * [Dashboard] Retry visbuilder dashboard test (#823) Seeing it pass in video replays but occassionally doesn't pass. However, it doesn't pass with security enabled so I'd imagine it could be related to the refreshing of the session. Issue: opensearch-project/OpenSearch-Dashboards#4947 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * Add new tests for alerting dashboards (#832) Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update source element in import_vector_map_tab.spec.js (#844) * Update source element in import_vector_map_tab.spec.js Signed-off-by: Junqiu Lei <[email protected]> * Update source element in import_vector_map_tab.spec.js Signed-off-by: Junqiu Lei <[email protected]> --------- Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: [email protected] <[email protected]> * Use index pattern id to find page in import_vector_map_tab.spec.js (#847) Signed-off-by: Junqiu Lei <[email protected]> Signed-off-by: [email protected] <[email protected]> * Buffer `waitForLoader` before checking for icon (#857) (#859) Some async calls occur after navigating and sometimes navigating too quickly prevents the calls from having correct data and therefore the homeIcon is in a bad state. This adds the ability to buffer before checking for the homeIcon. Defaulted to 0 ms, but for `yarn cypress:run-with-security` configures WAIT_FOR_LOADER_BUFFER_MS to be 500 ms Issue resolved: opensearch-project/OpenSearch-Dashboards#5028 Signed-off-by: Kawika Avilla <[email protected]> (cherry picked from commit e7c7e59) Co-authored-by: Kawika Avilla <[email protected]> Signed-off-by: [email protected] <[email protected]> * Overwrite removed and moved to after in unit test Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> * Update dashboard.spec.js Signed-off-by: leanneeliatra <[email protected]> * Update dashboard_sample_data_spec.js remove unneeded change Signed-off-by: leanneeliatra <[email protected]> --------- Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: [email protected] <[email protected]> Signed-off-by: leanneeliatra <[email protected]> Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Sirazh Gabdullin <[email protected]> Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Tyler Ohlsen <[email protected]> Signed-off-by: manasvinibs <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: Qingyang(Abby) Hu <[email protected]> Signed-off-by: Lin Wang <[email protected]> Signed-off-by: Derek Ho <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Ashish Agrawal <[email protected]> Signed-off-by: Junqiu Lei <[email protected]> Co-authored-by: Yulong Ruan <[email protected]> Co-authored-by: Kawika Avilla <[email protected]> Co-authored-by: Sirazh Gabdullin <[email protected]> Co-authored-by: Tyler Ohlsen <[email protected]> Co-authored-by: Jovan Cvetkovic <[email protected]> Co-authored-by: Manasvini B Suryanarayana <[email protected]> Co-authored-by: Kristen Tian <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Qingyang(Abby) Hu <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Lin Wang <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Joshua Li <[email protected]> Co-authored-by: Derek Ho <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ashish Agrawal <[email protected]> Co-authored-by: Junqiu Lei <[email protected]> (cherry picked from commit 21bf25f)

Change to toast message in Reports download spec

1fb1985

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

kavithacm requested a review from a team as a code owner February 24, 2023 21:55

joshuali925 reviewed Feb 24, 2023

View reviewed changes

cypress/integration/plugins/reports-dashboards/04-download.spec.js Outdated Show resolved Hide resolved

This was referenced Feb 28, 2023

Release version 2.6.0 opensearch-project/dashboards-reporting#63

Closed

[RELEASE] Release version 2.6.0 opensearch-project/opensearch-build#3081

Closed

Update test

8b9b01b

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

kavilla reviewed Mar 10, 2023

View reviewed changes

cypress/integration/plugins/reports-dashboards/04-download.spec.js Outdated Show resolved Hide resolved

ananzh added v2.10.0 backport 2.x labels Jul 10, 2023

kavilla approved these changes Jul 19, 2023

View reviewed changes

kavilla added the backport 2.9 label Jul 19, 2023

anirudha mentioned this pull request Jul 19, 2023

[RELEASE] Release version 2.9.0 opensearch-project/opensearch-build#3616

Closed

42 tasks

ashwin-pc approved these changes Aug 29, 2023

View reviewed changes

Fix lint

ddc1cd8

Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>

kavithacm requested review from xluo-aws, Hailong-am, SuZhou-Joe, ruanyl, wanglam and raintygao as code owners August 29, 2023 21:43

ruanyl approved these changes Aug 30, 2023

View reviewed changes

ashwin-pc merged commit 46c9a48 into opensearch-project:main Aug 30, 2023

This was referenced Aug 30, 2023

[Backport 2.x] Change to toast message in Reports #805

Merged

[Backport 2.9] Change to toast message in Reports #806

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Change to toast message in Reports #578

Change to toast message in Reports #578

kavithacm commented Feb 24, 2023 •

edited

Loading

ashwin-pc commented Mar 1, 2023

kavithacm commented Mar 2, 2023

kavilla commented Mar 10, 2023

kavilla commented Mar 15, 2023

kavithacm commented Jul 19, 2023

ashwin-pc commented Aug 29, 2023

ashwin-pc commented Aug 30, 2023

Change to toast message in Reports #578

Change to toast message in Reports #578

Conversation

kavithacm commented Feb 24, 2023 • edited Loading

Description

Issues Resolved

Check List

ashwin-pc commented Mar 1, 2023

kavithacm commented Mar 2, 2023

kavilla commented Mar 10, 2023

kavilla commented Mar 15, 2023

kavithacm commented Jul 19, 2023

ashwin-pc commented Aug 29, 2023

ashwin-pc commented Aug 30, 2023

kavithacm commented Feb 24, 2023 •

edited

Loading